Skip to content

Live TV Configuration Options#790

Open
michaelcresswell wants to merge 9 commits intojellyfin:masterfrom
michaelcresswell:live-tv-configuration
Open

Live TV Configuration Options#790
michaelcresswell wants to merge 9 commits intojellyfin:masterfrom
michaelcresswell:live-tv-configuration

Conversation

@michaelcresswell
Copy link
Contributor

Changes

Add channel title content and guide channel identifier configuration options to Live TV. Show logo on the right and title text on the left when both are visible, to better align with other clients.

Issues

Fixes #788

@michaelcresswell michaelcresswell requested a review from a team as a code owner March 15, 2026 15:20
@michaelcresswell
Copy link
Contributor Author

New Configuration Options

image

The default value matches the client behavior since pre-3.0.0. Now users can instead show the channel number portion only or the name portion only if desired.

Note: The title is shown in the OSD as well


image

The default values matches the client behavior since 3.1.6. Logo only is how the client behaved 3.0.0-3.1.5. Title only is a new possibility.

Sample Screenshots

A few possible configurations

Default settings (3.1.6+ Behavior)

image

Recreating 3.1.5 Behavior

image

Number-Only Title & Hidden Logo

image

@Alliyance
Copy link

Thank you. Pretty sure that will help. 🤞🏻

Copy link
Collaborator

@jimdogx jimdogx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing default english string in en_US translations file. That will get picked up and then translated to other languages for us.

@michaelcresswell
Copy link
Contributor Author

Good catch, I just added them

@@ -0,0 +1,5 @@
enum LiveTVChannelIdentifier
LOGO_AND_TITLE = "logoTitle"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency. We don't use snake case in any other enums, only uppercase.

Suggested change
LOGO_AND_TITLE = "logoTitle"
LOGOANDTITLE = "logoTitle"

@@ -0,0 +1,5 @@
enum LiveTVChannelTitle
NUMBER_AND_NAME = "numberName"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NUMBER_AND_NAME = "numberName"
NUMBERANDNAME = "numberName"

"author": "gabeluci"
},
{
"description": "Add channel title content and guide channel identifier configuration options to Live TV",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Front load the word setting so users know this is a new setting.

"Add setting to configure live TV channel title and logo..."

m.top.id = json.id
if isValid(json.number)
m.top.title = `${tr("CH")} ${json.number} ${json.name}`
if isValid(json.number) and not isStringEqual(channelTitleContent, LiveTVChannelTitle.NAME)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inverse the condition so we don't test json.number when it is LiveTVChannelTitle.NAME

Suggested change
if isValid(json.number) and not isStringEqual(channelTitleContent, LiveTVChannelTitle.NAME)
if not isStringEqual(channelTitleContent, LiveTVChannelTitle.NAME) and isValid(json.number)

"description": "Settings relating to Live TV.",
"children": [
{
"title": "Channel Title Content",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"title": "Channel Title Content",
"title": "Channel Title Info",

"children": [
{
"title": "Channel Title Content",
"description": "What to use for the channel title.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"description": "What to use for the channel title.",
"description": "Select what information to use as a channel's title.",

]
},
{
"title": "TV Guide Channel Identifier",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"title": "TV Guide Channel Identifier",
"title": "TV Guide Channel Display",

},
{
"title": "TV Guide Channel Identifier",
"description": "What to use for identifying a channel in the Live TV guide.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"description": "What to use for identifying a channel in the Live TV guide.",
"description": "Select how to display a channel in the live TV guide.",

Copy link
Member

@1hitsong 1hitsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Great job.

@jimdogx will still need to complete his review before merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

As of Version 3.1.7 the EPG display has the channel ID in each channel line. Unpleasent display.

4 participants